widget: Initialize a local variable

We will later pass this variable on to some other functions, so be safe
against them using the value.

aboutdialog: Only close on delete events

Clicking on the Credits button should not close the dialog.

fontchooswerwidget: Center spinbutton next to scales vertically

gl renderer: Render GL textures upside down

These come from an FBO so we need to flip the texture coords on the y
axis, just like we do everywhere else.

Revert "inspector: Stop adding event controllers to the object tree"

This reverts commit 8f3cb2658a79d20912b6f2ec96bc7e6bbdaeb067.

This didn't quite work out, since being in the object tree is
a prerequisite for showing up in the property editor.

Document new texture api

Use the new api to release resources

When we unrealize a GL area with outstanding textures,
call gdk_texture_release_gl on them to avoid later crashes.

Add a way to release GL resources

The inspector may hold on to render nodes and textures
beyond the lifetime of the widget (and thus the GL
resources). To handle this situation, allow the widget
to explicitly release the GL resources, and make
the texture available on the clent-side as a cairo
surface. This lets the recorder still show the content
after the widget is gone.

glarea: Drop the dispose vfunc

This was causing us to leak, in the following scenario:
1) gtk_widget_destroy is called on a GL area
2) dispose is run and clears the context
3) the GL area is unrealized, but the context is already cleared,
   so we leak all the GL buffers

glarea: Always delete textures on unrealize

We need to make the context current for deleting
textures as well, so just do it unconditionally.

glarea: Avoid a crash

We must only delete the texture if it is not NULL.

gdkgears: add a way to remove gears

This is needed to test the GL resource cleanup
code paths.

glarea: Add a pool for GL textures

Handle the situation that a GL texture might remain
in use (e.g. by a slow frame, or by the recorder)
In that case, we can't modify it but must use a
new one. Keep a pool of GL textures for this eventuality.

inspector: Don't leak recordings

g_list_model_get_item is transfer full, so we need
to drop the references we get from it. This was showing
up while testing the GL texture cache in GtkGLArea.

rendernodeimpl: Fix a crash

We may get a NULL renderer here.

applicationwindow: Fix measure implementation

Really calculate the menubar height, not the width.

sizerequest: Fix for_size adjustment

We need to pull the proper size from the size request cache and adjust
it accordingly.

box: Simplify some code

All out varibles in measure implementations must be non-null and -1 is
the default value for the baseline parameters.

box: Remove self assignment

box: Remove useless if statement

We already check right before this one whether child->pack != packing
and if so, we continue to the next iteration. So, no need to check again
whether the inverted condition child->pack == packing is true, because
it is.

label: Update layout width directly form allocation

For the one update_layout_width call in size_allocate, we can just use
the passed-in allocation width instead of a separate (relatively slow)
gtk_widget_get_width call.

widget: Clip redrawn area to widget clip

The clip can already be modified in size_allocate, so we never want a
widget to invalidate more than its actual clip region.

box: Remove some unneeded checks

We only call this function from the measure implementation and we should
never get NULL pointers there.

box: Remove an unnecessary local variable

box: Use GtkWidget's widget list in get_size

listbox: Don't assume that row child == focus widget

With widgets supporting child widgets and custom GtkListBoxRow
subclasses, this might not be true.

widget: Remove reparent special case

Since gtk_widget_reparent doesn't exist anymore, we can also remove this
special case.

container: Remove some unused includes

viewport: Remove some unused includes

gl area: Avoid trivialities

If we have zero pixels, no need to snapshot anything,
and produce GL errors while doing so.

gtk-demo: Add gears to the bowl

Just for fun.

Drop GtkGLArea::has-alpha

We now always behave as if has-alpha is TRUE.

Update all callers.

Convert gtkglarea to use snapshots

Just append a texture node.

Note that this is not 100% done yet. The GL area really
needs to keep a pool of textures, and only reuse them
once the GdkTexture object is gone.

gsk: Handle GL textures

These textures already have a GL texture in them,
no need to create a new one.

gdk: Add a gl texture implementation

This will be used to pass a GL textures from the application
(or rather, GtkGLArea) down to the GSK GL renderer.

Drop removed api from the docs

This should have been part of the previous commit.

gdk: Drop some unused cursor apis

The query function for cursor sizes and capabilities
are not very interesting. At least, they are not used
in GTK+, and all backends but X11 just hardcode
made-up values anyway. So, lets drop them.

inspector: Stop using gdk_display_get_default_cursor_size

If the cursor theme size setting has no value, just hardcode 32.
Also, allow ridiculously large cursors, just for fun.

tooltip: Stop using gdk_display_get_default_cursor_size

It makes more sense to use the actual cursor theme size here.

testgtk: Stop using gdk_display_get_maximal_cursor_size

This function is going away. Just make a reasonable assumption
that cursor sizes up to 128 are supported.

gtk/queryimmodules.c: Make the output deterministic.

Whilst working on the Reproducible Builds effort [0], we noticed that
queryimmodules generates non-reproducible output as it iterates over the
filesystem without sorting.

Patch attached.

 [0] https://reproducible-builds.org/

Signed-off-by: Chris Lamb <lamby@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786528

tree model sort: Fix set_sort_column

We were failing to change the sort order for the
default sort column in some cases. Fix that, and
add a testcase for this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=792459

Add a testcase for the previous fix

tree model sort: Fix initial default sort func

gtk_tree_sortable_has_default_sort_func should return
FALSE initially.

Fix the icontheme test

We rely on log messages here. Since logging is per-display
now, we need to set a display on our custom icontheme object
to get the expected log messages.

mountoperation: Fix a crash

We only create the gesture when the dialog is needed,
so don't free it unconditionally.

Drop ::button-press/release event

We are no longer using these, and there are several gestures
that can be used instead. If you need to catch raw button events,
use the ::event signal.

popover: Stop using ::button-press/release-event

Just use a generic ::event handler.

menushell: Stop using ::button-press/release-event

Merge the handlers into the existing handler for ::event.

menu: Remove stray references to 'button_press'

No code change, but this string was still used in
docs and comments.

places sidebar: Stop using ::button-press/release-event

We'll use a generic ::event signal instead.

gtk-demo: Stop using ::button-release-event

tests: Stop using ::button-press/release-event

The generic ::event signal works fine here.

testsuite: Stop using ::button-press-event

We can use the generic ::event signal instead.

tests: Stop using ::button-press-event

A gesture will do here.

Drop ::motion-notify-event

We no longer emit this signal. You can use various gestures
and event controllers instead. If you need to catch raw
motion events, use the generic ::event signal.

places sidebar: Stop using ::motion-notify-event

We can use the generic ::event signal here.

docs: Stop referring to specific event signals

These are going away.

gtk-demo: Stop using ::motion-notify-event in the hypertext demo

We already have a generic ::event handler, just use it for this.

tests: Stop using ::motion-notify-event

We can use the generic ::event signal here.

treeview: Stop using ::button-press-event

We can just use a gesture here.

recent chooser: Stop using ::button-press-event

We can just use a multipress gesture here.

mount operation: Stop using ::button-press-event

A multipress gesture does just fine here.

testinput: Port to a drag gesture

This replaces both button-press-event and motion-notify-event
handlers.

entry completion: Drop another ::button-press-event handler

This one will be replaced by a generic ::event handler.

docs: Stop referring to legacy event signals

These are going away.

entry completion: Stop using ::button-press-event

Just use the existing ::row-activated signal on the treeviews.

gtk-demo: Stop using ::button-press-event

Use gestures instead.

testgtk: Stop using ::button-press-event

This one example can just as well use a gesture.

places view: Stop using ::button-press-event

Instead, use a multi-press gesture in the row widget,
and emit ::popup-menu from there.

file chooser: Use a multipress gesture

This replaces the use of ::button-press-event. There's two
issues with this commit:
1) We don't have a good way to do the equivalent of
   gdk_event_triggers_context_menu with gestures
2) We have to defer to and idle to avoid ordering
   issues with the treeviews own gestures

Revert "file chooser: Allow activating without double-click"

This reverts commit fb0a13b7f070a14312dafa1e4df6ba03cf33be01.

It is getting in the way of gesture conversion, and didn't
really make anybody happy anyway.

app chooser: Use a gesture

Instead of connecting to ::button-press-event, use
a multipress gesture here.

Remove a leftover icon-size property

This made the 'style classes' demo crash.

Drop ::focus-in/out-event

These signals are no longer used in GTK+.
If you need to catch focus events, use the ::event signal.

treeview: Stop using ::focus-out-event

We can just use the generic ::event signal.

file chooser entry: Stop using ::focus-out-event

Just use the generic ::event signal.

spin button: Stop using ::focus-out-event

Just use the generic ::event signal.

calendar: Stop using ::focus-out-event

Use the generic ::event signal.

color editor: stop using ::focus-out-event

Again, the has-focus property is sufficient.

text cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

spin cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

combo cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

tests: Stop using ::focus-in/out-event

The has-focus property works just as well here.

popover: Stop using ::focus-in/out-event

We can use the is-active property of the window instead.

application: Stop using ::focus-in-event

We are really interested in the active window, so use
that property directly.

window: Stop using ::focus-in/out-event

We can just use the generic ::event.

text view: Stop using ::focus-in/out-event

We can just use the generic ::event here.

entry completion: Stop using ::focus-out-event

We can just use ::event.

entry: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

a11y: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

Drop gtk_widget_hide_on_delete

This function is misnamed - it is only ever relevant for
windows. And with the ::hide-on-delete property , it is
no longer necessary to use the signal for this simple case.

Drop ::event-after

This signal is not used in GTK+, and we are reducing the
amount of widget signals devoted to events in favor of
event controllers.

Drop ::grab-broken-event

This signal is no longer used in GTK+. Grab broken events
can be handled with the generic ::event signal.

button: Stop using ::grab-broken-event

The event-specific signals are going away. Just use ::event here.

menushell: Stop using ::grab-broken

The event-specific signals are going away. Just use ::event here.

notebook: some cleanup

notebook: Use a motion event controller

This lets us avoid the ::motion-notify-event handler.

textview: Use a motion event controller

This lets us replace the ::motion-notify-event handler.

Drop ::enter/leave-notify-event

These signals are no longer used in GTK+ and have a (not quite
perfect yet) replacement with GtkEventControllerMotion.

If you need to catch the raw events, the generic ::event
signal still works.

menu: Use a motion event controller

This lets us replace the ::enter/leave-notify-event handlers.
Not that there is a FIXME here - we currently can't get the
crossing mode from the event controller, so we have to fall
back to gtk_get_current_event().